[DowngradePhp84] Add DowngradeExitNamedArgumentRector - #387

@TomasVotruba ready 👍

LGTM 👍 Probably all named args in native functions should be downgraded, not just exit()

That's special on exit/die only for php 8.4.

Other downgrade named argument is on DowngradePhp80 DowngradeNamedArgumentRector

The DowngradeNamedArgumentRector should handle the exit() as well

Why this doesn't worK? https://3v4l.org/f0f3a#v8.3.33
Weird :D

that's "Process exited with code 1" means works on php 8.4, the page shows 2 part,

php 8.4: Process exited with code 1.
php 8.3: syntax error, unexpected token ":", expecting ")"

see https://php.watch/versions/8.4/exit-die-as-functions

the transformation is remove the identifier status.

DowngradeNamedArgumentRector will automatically handle exit(status: 1), but if user only want to downgrade to php 8.3, it will not catch without this rule.

The downgrade process is from top to bottom, if I want to downgrade to php 8.3 only, this take care of it.

The original node is Expr\Exit_ https://github.com/nikic/PHP-Parser/blob/fbd47f7ebcbb450138d92642a0a53b72a5285dda/lib/PhpParser/Node/Expr/Exit_.php#L7

but start from php 8.4, it can be any function call with status named arg https://github.com/nikic/PHP-Parser/blob/fbd47f7ebcbb450138d92642a0a53b72a5285dda/lib/PhpParser/Node/Expr/FuncCall.php#L8

I see, so PHP 8.4 switched from exit type to normal function. Thanks 👍